Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use timezone.now() instead of datetime.today() to determine time #5

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

johanneswilm
Copy link

Django 1.4 requires datetime instances to be time zone aware by default.

Django 1.4 requires datetime instances to be time zone aware by default.
@johanneswilm
Copy link
Author

btw, the API seems to be outdated. You guys switched to a very different model, but the docs are still the same. Is there anywhere I can see how to do the stuyff that is being done in the API?

@RobCombs
Copy link
Owner

RobCombs commented Jan 7, 2013

Not at the moment…sorry. The API docs are indeed outdated and they don't reflect the new model that is being used. You can look at your console to see which http requests are being executed by django-locking.

That will give you a sense as to how the ajax calls are communicating with the views. From that point, you can deduce the api calls that are needed for your purpose. We'll gladly accept a patch to get those docs updated once you get that figured out.

Thanks!

RobCombs | Sr. Software Engineer, technology | CMGdigital
[email protected] | p: 678-645-4185 | m: 770-377-2732

From: johanneswilm <[email protected]mailto:[email protected]>
Reply-To: RobCombs/django-locking <[email protected]mailto:[email protected]>
Date: Sat, 5 Jan 2013 07:07:44 -0800
To: RobCombs/django-locking <[email protected]mailto:[email protected]>
Subject: Re: [django-locking] use timezone.now() instead of datetime.today() to determine time (#5)

btw, the API seems to be outdated. You guys switched to a very different model, but the docs are still the same. Is there anywhere I can see how to do the stuyff that is being done in the API?


Reply to this email directly or view it on GitHubhttps://github.com//pull/5#issuecomment-11915016.

@johanneswilm
Copy link
Author

I believe this last addition should make ti compatible with django 1.3

@johanneswilm
Copy link
Author

btw, I gave up on django-locking and moved on to django-simple-lock. They just updated their documentation.

@RobCombs
Copy link
Owner

RobCombs commented Jan 7, 2013

Sounds good. Thanks for the update!

From: johanneswilm <[email protected]mailto:[email protected]>
Reply-To: RobCombs/django-locking <[email protected]mailto:[email protected]>
Date: Mon, 7 Jan 2013 07:55:06 -0800
To: RobCombs/django-locking <[email protected]mailto:[email protected]>
Cc: Rob Combs <[email protected]mailto:[email protected]>
Subject: Re: [django-locking] use timezone.now() instead of datetime.today() to determine time (#5)

btw, I gave up on django-locking and moved on to django-simple-lock. They just updated their documentation.


Reply to this email directly or view it on GitHubhttps://github.com//pull/5#issuecomment-11956805.

@houdelou
Copy link

Hello,

I got an error that I think is related to this issue. And just git clone the repository. Using Django 2.4 and Python 2.7.

can't subtract offset-naive and offset-aware datetimes

django_locking-0.3.2-py2.7.egg/locking/models.py in is_locked, line 88

Thank you

Louis

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants